Skip to content

[Oracle] Support for INSERT INTO (<sub-query>) ...#2276

Merged
iffyio merged 9 commits intoapache:mainfrom
xitep:main
Mar 20, 2026
Merged

[Oracle] Support for INSERT INTO (<sub-query>) ...#2276
iffyio merged 9 commits intoapache:mainfrom
xitep:main

Conversation

@xitep
Copy link
Contributor

@xitep xitep commented Mar 10, 2026

No description provided.

///
/// If `full_query == true` attempt to detect a full query with its
/// optional, leading `WITH` clause, e.g. `(WITH ...)`
fn peek_subquery_start(&mut self, full_query: bool) -> bool {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we maybe add a separate function peek_cte_or_subquery (maybe better naming) that handles the WITH case? i.e.

return self.peek_token(LPAREN, WITH) || self.peek_subquery_start()

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the two methods are simpler. 👍 but it ended up longer than you suggested :/

@xitep xitep requested a review from iffyio March 13, 2026 16:23
Copy link
Contributor

@iffyio iffyio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks @xitep!

@iffyio iffyio added this pull request to the merge queue Mar 20, 2026
Merged via the queue into apache:main with commit 50921b1 Mar 20, 2026
10 checks passed
@xitep
Copy link
Contributor Author

xitep commented Mar 20, 2026

thanks @iffyio

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants